@clue any thoughts about this?

@HLeithner Thank you very much for bringing this up, I think this is an excellent starting point to discuss a DIC integration in X!

I like the direction this is taking and agree that adding an optional DIC as the first argument makes perfect sense. I also agree that this should allow swapping out internal implementation details such as the socket server and eventually the logger.

To move things forward, I would suggesting starting with a smaller changeset and possibly remove the logging aspect from this PR to fully focus on the DIC part. Injecting a custom logger makes perfect sense, but I think it makes sense to address separate implementation approaches in a separate PR.

Once the DIC is in, I'm also really looking forward to using this for controller instantiation to automatically inject required dependencies and only instantiate controllers when they're actually needed. This is going to be an interesting challenge to make sure this will be done on demand behind existing SAPIs and probably on-load when using the built-in webserver.

Full DIC integration is definitely on the mid-term roadmap and this is something I'm planning to look into in the next few weeks/months. If you feel this is something you can contribute to, make sure let me know 👍

To move things forward, I would suggesting starting with a smaller changeset and possibly remove the logging aspect from this PR to fully focus on the DIC part. Injecting a custom logger makes perfect sense, but I think it makes sense to address separate implementation approaches in a separate PR.

Ok will remove the logger and move it to it's own PR.

Once the DIC is in, I'm also really looking forward to using this for controller instantiation to automatically inject required dependencies and only instantiate controllers when they're actually needed. This is going to be an interesting challenge to make sure this will be done on demand behind existing SAPIs and probably on-load when using the built-in webserver.

So you would like to have some magic that boot a controller (or mvc "component") when required in a request. Isn't this done already by fast-route? I didn't checked the code but if I use

        $app->get(
            '/messages',
            'My\Messages\Controller::getMessages'
        );

I would expect the Controller class will only loaded when /messages is requested. ymmv

Full DIC integration is definitely on the mid-term roadmap and this is something I'm planning to look into in the next few weeks/months. If you feel this is something you can contribute to, make sure let me know 👍

Of course I will help on this topic.

For this PR, would you like a b/c layer for $loop in the App constructor?

What's about the socket Server Context? should be move to it's own PR too?